Skip to content

doc: update AUTHORS list#1586

Merged
rvagg merged 1 commit into
nodejs:masterfrom
rvagg:update-authors
May 4, 2015
Merged

doc: update AUTHORS list#1586
rvagg merged 1 commit into
nodejs:masterfrom
rvagg:update-authors

Conversation

@rvagg

@rvagg rvagg commented May 2, 2015

Copy link
Copy Markdown
Member

Update AUTHORS list using tools/update-authors.sh

Is Guilherme Souza <sitegui@sitegui.com.br> the same person as Guilherme de Souza <gui_souza@me.com>? / @sitegui / #1493

@sitegui

sitegui commented May 2, 2015

Copy link
Copy Markdown
Contributor

No, @rvagg, thanks for asking :)

@mscdex mscdex added the doc Issues and PRs related to the documentations. label May 2, 2015
@chrisdickinson

Copy link
Copy Markdown
Contributor

LGTM

Update AUTHORS list using tools/update-authors.sh

PR-URL: nodejs#1586
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
@rvagg rvagg force-pushed the update-authors branch from ee19954 to dacc1fa Compare May 4, 2015 03:31
@rvagg rvagg merged commit dacc1fa into nodejs:master May 4, 2015
@rvagg

rvagg commented May 4, 2015

Copy link
Copy Markdown
Member Author

landed @ dacc1fa

@rvagg rvagg mentioned this pull request May 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants